Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added docs explaining CMS code.json #5

Merged
merged 12 commits into from
Feb 7, 2025
Merged

Added docs explaining CMS code.json #5

merged 12 commits into from
Feb 7, 2025

Conversation

natalialuzuriaga
Copy link
Contributor

@natalialuzuriaga natalialuzuriaga commented Feb 6, 2025

Problem

This new repository does not contain information about CMS code.json metadata standard.

Solution

Created the following items:

  • metadata.md provides an Overview on CMS code.json metadata standard with a table of all the fields and information on requesting new fields to be added.
  • procedures.md details procedures and tools that assist with creating and updating metadata. References repo-scaffolder, form site, and automated-codejson-generator (once @sachin-panayil completes this!).
  • metadata-field-addition.md is an issue template used for requesting new fields
  • Added repository information to all markdown files (README.md, CODEOWNERS.md, etc.)

@natalialuzuriaga natalialuzuriaga marked this pull request as ready for review February 7, 2025 00:36
<td>1, 2, 3, 4</td>
</tr>
<tr>
<td>contractNumber</td>
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Open to any ideas on ordering the new fields in the schema!

Copy link
Collaborator

@sachin-panayil sachin-panayil left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

dope work and is easy to read 🚢 🚀

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is clean!!! nice job @natalialuzuriaga

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

will add on this when i finish with my docs

Copy link
Contributor

@decause-gov decause-gov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM +1 🚢

@decause-gov decause-gov merged commit efc89c2 into main Feb 7, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants